-
-
Notifications
You must be signed in to change notification settings - Fork 459
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[IMP] oca_publish_modules: tool for publishing modules #409
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Interesting. requires a bit of documentation.
cc @sbidoul @pedrobaeza
4d55001
to
ad4c185
Compare
bd1676e
to
9e411b6
Compare
I have improved/fixed the flow according my research, but thanks for initial idea, @alexey-pelykh ! |
9e411b6
to
ad4b257
Compare
This is now completed and green. |
Good news! Rémi Rahir from Odoo has told me several tricks for improving this through JSON endpoints, so I will try soon to implement these suggestions. |
@pedrobaeza at what stage are we on this one? |
Stalled |
@pedrobaeza maybe let's just continue the work w/o those APIs? |
Now it's in @simahawk's hands |
@alexey-pelykh @pedrobaeza is not in my hands: if this is something that is working, please, feel free to finish it. |
@simahawk it was working, then there was a hope for APIs, but I'm not sure if that progressed anywhere. My vision is to rebase this, check that it's still working and do the magic of adopting it. But I need someone to help testing, @pedrobaeza helped me last time and contributed a lot to the original PR |
Sorry, but I'm not able to do it. Too many hours last time and having our own apps store, I'm personally not making any attention to Odoo's one. |
@pedrobaeza that makes perfect sense, yet I don't really have the means to test it, so unless someone with means will assist (anyone, really), I won't be able to push this forward. |
@alexey-pelykh I'm not a huge fan of the Odoo app store either but I think it's important OCA is present there too. if you rebase we'll find a way to test it. Either myself, @simahawk or @gurneyalex I guess. |
@sbidoul neither am I, but that's a free marketing :D sure, I'll rebase and let's make some tests |
eb50812
to
13493d7
Compare
same story here: if I land on odoo.com is because of clicks on search engine results or shared links 😄 |
@alexey-pelykh ciao :) Have you found time to make some tests? |
13493d7
to
96b06eb
Compare
@alexey-pelykh thanks a lot for this. I tweaked it a bit to make it work in the current state of the universe: brainbeanapps#1 I published the current status at https://odoo-community.org/apps.odoo.com/ |
Make it work with current apps.odoo.com. - ssh/https url support - avoid multi-page search results on apps.odoo.com - some scraper update to match current apps.odoo.com Assume chromedriver is in path (could not make it work otherwise)
@alexey-pelykh you you mind giving OCA/maintainer-tools maintainers write access to this branch? |
Co-authored-by: Stéphane Bidoul <[email protected]>
@sbidoul I would love to, but no such checkbox for me :( |
Ok, I'll merge then, and we can iterate from here. Thanks a lot! |
@pedrobaeza